Skip to content

Conversation

@Dmatrushka19
Copy link
Contributor

Изменения

Добавил подтверждение отвязки метода авторизации.

Детали реализации

После нажатия на один из методов, который необходимо отвязать, всплывает окно с двумя кнопками "отвязать" и "не отвязывать". После нажатия на первую кнопку, происходит отвязка метода предыдущим способом с проверкой на единственность (я ее оставил как было), после нажатия на вторую - просто закрытие модального окна.

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@Dmatrushka19 Dmatrushka19 requested a review from BatuevIO January 28, 2025 12:22
@Dmatrushka19 Dmatrushka19 self-assigned this Jan 28, 2025
@Dmatrushka19 Dmatrushka19 added the good first issue 👶 Задача подходит для новичков label Jan 28, 2025
@Dmatrushka19 Dmatrushka19 changed the title Add confirmation delete login method Add confirmation delete auth method Jan 28, 2025
@Dmatrushka19 Dmatrushka19 changed the title Add confirmation delete auth method Add confirmation delete auth methods Jan 28, 2025
@Dmatrushka19 Dmatrushka19 changed the title Add confirmation delete auth methods Add confirmation for delete auth methods Jan 28, 2025
@Dmatrushka19
Copy link
Contributor Author

В последнем (третьем коммите) поменял между собой цвета кнопок: "отвязать" написано обычным черным цветом, "не отвязывать" - как отмена операции выделено красным цветом.

@Dmatrushka19
Copy link
Contributor Author

Перенести variant="flat" size="large" class="oauth-button" в IrdomAuthButton для login стиля, для аuth-edit тоже что-нибудь придумать

@Dmatrushka19
Copy link
Contributor Author

ДОБИЛ ЗАДАЧУ! Стили у кнопочек при первой аутентификации теперь как были изначально и добавилась менюшка подтверждения отвязки метода.

Филипенков Дмитрий Алексеевич added 4 commits February 18, 2025 18:10
…rdomAuthButton.vue в AuthView. Не помогло поменять стили кнопок при отвязке, надо думать дальше.
…ровано системой если обращаться без props
@BatuevIO BatuevIO merged commit 2f6ce44 into main Feb 20, 2025
2 checks passed
@BatuevIO BatuevIO deleted the add-confirmation-delete-login-method branch February 20, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue 👶 Задача подходит для новичков

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Добавить подтверждение отвязки метода входа

3 participants